Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1043 +/- ##
==========================================
+ Coverage 89.82% 90.98% +1.16%
==========================================
Files 25 25
Lines 4391 4393 +2
==========================================
+ Hits 3944 3997 +53
+ Misses 447 396 -51 ☔ View full report in Codecov by Sentry. |
akx
approved these changes
Nov 18, 2023
Member
akx
left a comment
There was a problem hiding this comment.
Looks straightforward! Thank you for putting this work in 👍
Contributor
Author
|
Thanks! Once released, this will be very helpful for https://github.com/hedyorg/hedy! (Speaking of, any idea when that release might be? 😇🙏) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This upgrades the CLDR database to release 43.
I tried upgrading to 44 initially, but there seemed to be a lot of breaking changes in there that I didn't know how to deal with, and the locales I'm personally interested in are already supported in 43.
I had to make two additional changes in addition to updating the import:
One doctest, regular spaces have been replaced by thin spaces
In the importing of
parentLocales. The newsupplementalData.xmlcontains a new type of declaration:This refers to a locale called
sr_ME, but there is no.xmlfile for it and so no.datfile gets generated either. At runtime, when the locale forsr_Cyrl_MEis looked up, we try to merge the data fromsr_MEinto it and then get aFileNotFoundexception whensr_ME.datdoesn't exist.From the description of this new feature:
I figured that since this type of information is new and CLDR-42 didn't have it yet, it wouldn't hurt to just ignore it for now. We don't get the benefit of the new inheritable information, but we don't break either and at least we'll be able to consume the new core data for new locales.
This change adds support for the following new locales: